Skip to content

#51: Add unit test for self-contact - #52

Open
cwschilly wants to merge 13 commits into
sandialabs:mainfrom
NexGenAnalytics:add-self-contact-test
Open

#51: Add unit test for self-contact#52
cwschilly wants to merge 13 commits into
sandialabs:mainfrom
NexGenAnalytics:add-self-contact-test

Conversation

@cwschilly

Copy link
Copy Markdown
Collaborator

Fixes #51

nmm0 added a commit to NexGenAnalytics/distBVH that referenced this pull request Apr 24, 2025
@cwschilly cwschilly self-assigned this May 6, 2025
@cwschilly
cwschilly marked this pull request as ready for review May 6, 2025 14:11
@cwschilly
cwschilly requested review from cz4rs, nmm0 and pierrepebay May 6, 2025 14:11
@cwschilly

Copy link
Copy Markdown
Collaborator Author

The Integration Tests failure will be resolved by #26

@cz4rs cz4rs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@cwschilly
cwschilly force-pushed the add-self-contact-test branch from a04fb3f to 2b48f12 Compare January 15, 2026 19:44
@cwschilly

Copy link
Copy Markdown
Collaborator Author

New failures after rebasing 😄

Looking into the cause now

@cwschilly
cwschilly force-pushed the add-self-contact-test branch from a82aea1 to 0650ae9 Compare January 22, 2026 17:34
@cwschilly

Copy link
Copy Markdown
Collaborator Author

Just a faulty rebase on my part (improperly resolved some merge conflicts). Should be good to go now @nmm0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add unit test for self-contact

3 participants